🐛 Critical Bug: Silent TEE Failover Failure

File: src/opengradient/client/tee_connection.py — RegistryTEEConnection.reconnect()

What's broken

The try/except Exception block in reconnect() accidentally wraps the entire body including self._active = self._connect(). If _connect() raises (registry down, no active TEE, TLS error), the exception is silently swallowed at DEBUG level and self._active is never updated.

The except was intended to guard old-client cleanup (matching StaticTEEConnection), not the reconnect itself.

Before (buggy):

async with self._refresh_lock:
    try:
        self._active = self._connect()  # ← failure here is silently eaten
    except Exception:
        logger.debug("Failed to close previous HTTP client...")  # wrong scope

After (fixed):

async with self._refresh_lock:
    old_client = self._active.http_client
    self._active = self._connect()  # ← now propagates on failure
    try:
        await old_client.aclose()  # ← only cleanup is guarded
    except Exception:
        logger.debug("Failed to close previous HTTP client...")

Impact

Verification

Matches the pattern already used correctly in StaticTEEConnection.reconnect() (L78–L85).